Skip to content

fix(issue): use correct fallback for unrecognized alias-suffix inputs#72

Merged
BYK merged 1 commit intomainfrom
fix/issue-view-short-id-fallback
Jan 26, 2026
Merged

fix(issue): use correct fallback for unrecognized alias-suffix inputs#72
BYK merged 1 commit intomainfrom
fix/issue-view-short-id-fallback

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Jan 26, 2026

Fixes issue ID resolution for standard short IDs like CRAFT-G when the prefix is not a known alias. The fallback now correctly calls resolveFullShortId instead of resolveShortSuffixId.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 26, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Issue

  • Add workspace-scoped alias cache by BYK in #52
  • Add short ID aliases for multi-project support by BYK in #31

Other

  • (api) Align with gh api and curl conventions by BYK in #60
  • (auth) Add press 'c' to copy URL during login flow by betegon in #58
  • (commands) Rename get commands to view and add -w browser flag by BYK in #53
  • Add explain and plan commands (Seer AI) by MathurAditya724 in #39
  • Add Sentry SDK for error tracking and usage telemetry by BYK in #63

Bug Fixes 🐛

Issue

  • Use correct fallback for unrecognized alias-suffix inputs by BYK in #72
  • Handle cross-org project slug collisions in alias generation by BYK in #62
  • Use org-scoped endpoint for latest event + enhanced display by betegon in #40

Other

  • (api) Use query params for --field with GET requests by BYK in #59

Documentation 📚

  • (readme) Add installation section by betegon in #65
  • Update command references from 'get' to 'view' and document -w flag by BYK in #54

Internal Changes 🔧

  • (release) Fix changelog-preview permissions by BYK in #41
  • Rename config folder from .sentry-cli-next to .sentry by BYK in #50

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 26, 2026

Codecov Results 📊

✅ Patch coverage is 100.00%. Project has 1775 uncovered lines.
❌ Project coverage is 62.56%. Comparing base (base) to head (head).

Files with missing lines (22)
File Patch % Lines
human.ts 18.35% ⚠️ 672 Missing
resolve-target.ts 8.11% ⚠️ 306 Missing
oauth.ts 21.71% ⚠️ 202 Missing
resolver.ts 3.23% ⚠️ 120 Missing
errors.ts 5.94% ⚠️ 95 Missing
api-client.ts 72.40% ⚠️ 77 Missing
fixture.ts 28.74% ⚠️ 62 Missing
api.ts 89.78% ⚠️ 47 Missing
seer.ts 75.54% ⚠️ 45 Missing
errors.ts 73.17% ⚠️ 33 Missing
seer.ts 77.04% ⚠️ 31 Missing
preload.ts 39.02% ⚠️ 25 Missing
detector.ts 87.79% ⚠️ 16 Missing
cache.ts 76.27% ⚠️ 14 Missing
telemetry.ts 89.87% ⚠️ 8 Missing
config.ts 97.11% ⚠️ 7 Missing
index.ts 95.06% ⚠️ 4 Missing
colors.ts 91.84% ⚠️ 4 Missing
env-file.ts 97.17% ⚠️ 3 Missing
alias.ts 98.56% ⚠️ 2 Missing
java.ts 97.22% ⚠️ 1 Missing
parser.ts 98.63% ⚠️ 1 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    63.80%    62.56%    -1.24%
==========================================
  Files           35        40        +5
  Lines         3956      4741      +785
  Branches         0         0         —
==========================================
+ Hits          2524      2966      +442
- Misses        1432      1775      +343
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK force-pushed the fix/issue-view-short-id-fallback branch from a6c2e95 to bba10b1 Compare January 26, 2026 20:34
@BYK BYK marked this pull request as ready for review January 26, 2026 20:44
@BYK BYK merged commit 1c1ee99 into main Jan 26, 2026
15 checks passed
@BYK BYK deleted the fix/issue-view-short-id-fallback branch January 26, 2026 20:56
MathurAditya724 added a commit that referenced this pull request Apr 9, 2026
Closes getsentry/cli-init-api#72

The server-side AI codemod planner generates .env.sentry-build-plugin with
an empty SENTRY_AUTH_TOKEN= because it never has access to the user's auth
token (by design — the token is only used as an HTTP Authorization header).

This change makes the CLI fill in the token client-side during patchset
application. When a created env file contains an empty SENTRY_AUTH_TOKEN=,
the locally-available auth token is injected before writing to disk. The
token never leaves the client.
MathurAditya724 added a commit that referenced this pull request Apr 10, 2026
…706)

## Summary
- Threads the auth token from `wizard-runner.ts` through to
`local-ops.ts` via a new `authToken` field on `WizardOptions`
- During patchset application, when the server creates an env file with
an empty `SENTRY_AUTH_TOKEN=`, the CLI injects the locally-available
auth token before writing to disk
- Token injection only applies to `.env*` files and only fills in empty
values — non-empty values are never overwritten
- The auth token never leaves the client (not sent to the server
workflow)

Closes getsentry/cli-init-api#72
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant